testsuite: Turn all CSS test ui files into regular windows

Use undecorated windows instead of popups.

testsuite: Make a11y tests use regular windows

Merge branch 'wip/otte/for-master' into 'master'

reftests: Change popup windows to non-decorated windows

See merge request GNOME/gtk!1435

reftests: Change popup windows to non-decorated windows

This allows getting rid of popup windows.

Merge branch 'zhaoqiang/gtk-To_avoid_compiler_warning'

Merge branch 'master-subtract-base-size' into 'master'

gdk: Subtract base size when checking aspect ratio

See merge request GNOME/gtk!1331

Merge branch 'wip/otte/for-master' into 'master'

slicelistmodel: Fix two wrong computations

See merge request GNOME/gtk!1433

slicelistmodel: Fix two wrong computations

Both of those are thinkos during math.
Found by Mohammed Sadiq.

Testcases triggering both have been added.

Force-create style contexts in realize

This is necessary as long as style contexts are
required for proper css change tracking.

Fixes #2435

Merge branch 'master' into 'master'

Fixed OpenGL extension detection for extensions promoted to OpenGL core.

Closes #2428

See merge request GNOME/gtk!1424

Fix detection of OpenGL 3.3 core GL_ARB_timer_query.

Some systems (notably macOS) will not allow enumeration of an extension that has been promoted to core OpenGL for context in use. This change assumes that GL_ARB_timer_query is available on OpenGL 3.3+.

I could not find definitive information on whether GL_ARB_debug_output or GL_KHR_debug have been added to core. Other extensions in use were addressed by https://gitlab.gnome.org/GNOME/gtk/merge_requests/1422 .

Fixed OpenGL extension detection for extensions promoted to OpenGL core.

For a given OpenGL context, macOS in particular does not support enumeration / detection of OpenGL features that have been promoted to core OpenGL functionality. It is possible other drivers are the same. This change assumes support for GL_ARB_texture_non_power_of_two with OpenGL 2.0+, GL_ARB_texture_rectangle with OpenGL 3.1+ and GL_EXT_framebuffer_blit with OpenGL 3.0+. I failed to find definitive information on whether GL_GREMEDY_frame_terminator has been promoted to OpenGL core, or whether GL_ANGLE_framebuffer_blit or GL_EXT_unpack_subimage have been promoted to core in OpenGL ES. This change results in a significant GtkGLArea performance boost on macOS.

Closes #2428

Merge branch 'ci-flatpak' into 'master'

CI pipeline changes for Flatpak jobs

See merge request GNOME/gtk!1429

ci: Remove G_MESSAGES_DEBUG

We don't need all the debugging messages.

ci: Allow flatpak jobs to fail

We have more accurate CI jobs, now; the Flatpak jobs are nice to have.

Merge branch 'gi-fixes' into 'master'

Various fixes for documentation and introspectable API

See merge request GNOME/gtk!1427

Make release builds mandatory for ci

The tests are now passing in release builds, lets keep
it that way.

testsuite: Handle icontheme test better

Skip just the failing tests in release builds.

gsk: Fix the compiler warning differently

Merge branch 'report-fixes' into 'master'

Fixes for the CI test reports

See merge request GNOME/gtk!1426

ci: Use per-suite anchors

Anchors are global, so they need to be namespaced.

ci: Rearrange the results in the HTML report

Move the failures up top, so they stand out.

ci: Fix the HTML anchor in the report

ci: Use result instead of exit code in the JUnit report

We have a result code coming from Meson which is more accurate than just
looking at the exit code of the unit.

ci: Unexpected passes are failures

List them as such in our reports.

ci: Put the branch name in the HTML header

ci: Move HTML report meta from header to article

Makes the bar smaller.

ci: Re-enable ccache

Remove gtk_style_context_get_parent()

The function was declared and mentioned in the documentation, but it has
been removed in commit 416182a20d062d5865916f86909354ec85d00dd1.

gi: Skip gtk_custom_layout_new()

The whole GtkCustomLayout class is mostly a C convenience API, and
there's no reason why high level languages should use it.

Fix the function name in the gtk-doc stanza

The documentation is still referring to the old function's name.

Match argument name between declaration and definition

Otherwise the documentation and the introspection data will complain.

docs: Match the argument name with its declaration

Add a notify function to GdkContentProvider

The callback-based content providers need a GDestroyNotify function to
free the data passed to them on construction, otherwise they are going
to leak.

Remove border-half-pixel reftest from xfail

It passes both locally and on ci now.

Merge branch 'wip/otte/no-clip-on-draw' into 'master'

Various fixes to rendernode drawing with Cairo

See merge request GNOME/gtk!1425

gdk: Remove gdk_cairo_get_clip_rectangle()

The function is fundamentally broken for unbounded surfaces.
If a surface is unbounded, we cannot represent this as a
cairo_rectangle_int_t, and using the return value doesn't work because
it's already used for something else.

In GTK3, unbounded surfaces aren't a problem, but GTK4 uses recording
surfaces.

So better remove that function before we keep using it and using it
wrong.

cairoblur: Get rid of gdk_cairo_get_clip_rectangle() call

It's about to be removed.

gsk: Clip shadow node before push_group()

We want to be sure to push a group that's as small as possible, so we
clip to the child's bounds, because that's the smallest we can go.

gsk: Add a utility function for rectangles

... and use it.

cairoblur: Move the check for early exit

That means we only have one place where we check all kinds of early
exits.

rendernode: Avoid rounding errors

Compute the pattern matrix directly instead of transforming the cairo_t.

This ensures that when node_size / texture_size is some obscure floating
point value, we don't get rounding issues from scaling by it once we
draw the texture_size rectangle.

I have no actual failure where this comes in handy, but I had written
the code anyway, so decided to keep it.

testsuite: Add tests for rounded rect code

... including the bug that was fixed in the parent commit.

roundedrect: Fix inlining of graphene functions

graphene treats equality for contains() operations as always matching,
so do the same thing.

This is because unlike integer math, floating point cannot do the "as
close as possible to the point, but not reaching it" operation that
integer does by just subtracting 1.

roundedrect: Fix gsk_rounded_rect_intersects_rect()

The previous code would return FALSE for
  gsk_rounded_rect_intersects_rect (&rounded, &rounded.bounds);
if rounded was indeed rounded.

testsuite: Don't use gdk_cairo_get_clip_rectangle()

I want to remove it.

testsuite: Don't destroy surfaces that are still used

Coercing the surfaces must not unref the old surface, because the old
surface is going to be saved to a file.

testsuite: Remove accidentally checked in file

This file should go in the listview branch I guess.

Fix flickery hover

Commit 47c44644b109e0 was a bit overzealous in fixing
compiler warnings. We still need to call collect_textures,
even if we don't need the number that it returns.

Merge branch 'wip/baedert/cairo-paint' into 'master'

Remove cairo_clip() when drawing render nodes

Closes #2431

See merge request GNOME/gtk!1418

Remove unused signal enum values

GtkWidget lost even more signals, so we don't need
these enum values anymore.

popovermenu: Avoid an uninitialized variable

Pointed out by Timm/clang.

Fixes https://gitlab.gnome.org/GNOME/gtk/issues/2434

Merge branch 'ci-pages' into 'master'

Rename the SCSS files for our themes

Closes #2423

See merge request GNOME/gtk!1423

Rename the SCSS files for our themes

It seems that Meson's gnome.compile_resources() cannot deal with two
files with the same name under different directories, which breaks the
build parallelism because the GResource file ends up not depending on
either the Adwaita or the HighContrast gtk-contained.css file.

This commit only changes the on-disk names of the Adwaita and
HighContrast SCSS files, and the corresponding generated CSS files; the
files in the GResource are going to be aliased to the old file names, to
minimise the breakage. We might want to change the theme entry points at
some later date, if we decide to commit to this naming scheme.

Fixes: #2423
See Meson bug: https://github.com/mesonbuild/meson/issues/6615

Fix the Emoji chooser finalize

If the Emoji chooser is finalized before the idle
has run, it would spew criticals, breaking some tests.
Avoid that.

Merge branch 'gbsneto/fix-emojichooser-header' into 'master'

build: Install gtkemojichooser.h

See merge request GNOME/gtk!1420

build: Install gtkemojichooser.h

It was made public at ef7172dc1ee86a, but the header file
wasn't added to the public headers list.

Add the header file to the list.

Merge branch 'ci-pages' into 'master'

Small CI pipeline fixes

See merge request GNOME/gtk!1419

ci: Update flatpak job

Use `flatpak build` and point at the nightly repo when generating the
bundle.

ci: Deploy the API ref before flatpak

ci: Fix linking of the CI assets in the report

The report is relative to the build directory, and so are the generated
assets.

testsuite: Warn if a necessary env var is missing

Better to warn if we know why the tests are
going to fail. Saves headscratching later.

Move performance tests to the right testsuite

This was a copy-paste error.

rendernode: Don't clip when _draw()ing

rendernode: draw() only in node bounds

We currently disable when draw()ing nodes using the cairo fallback path,
which means we can't just use cairo_paint(). Use a proper rectangle
instead.

Fixes #2431

Merge branch 'cleanup-profiler-marks' into 'master'

Cleanup profiler marks

See merge request GNOME/gtk!1417

profile: Use separate names for frameclock marks instead of using details

The marks are averaged based on the name, so this makes more sense.

Also rename the map/unmap marks to have the same capitalization as
everything else.

profiler: Add G_GNUC_PRINTF markers to silence warnings

I was getting CI failures like:
 ../gdk/gdkprofiler.c: In function ‘add_markvf’:
../gdk/gdkprofiler.c:111:3: error: function ‘add_markvf’ might be a candidate for ‘gnu_printf’ format attribute [-Werror=suggest-attribute=format]

profiler: Clean up profiler marks for frameclock

This drops the marks for before/after-paint as they are internal
things that very rarely use any time, and also flush/resume-events
as any events reported here will get separate marks so will be easy
to see anyway.

Also, we rename the entire frameclock cycle to "frameclock cycle"
rather than "paint_idle" which is rather cryptic.

profiler: Add profiler marks for when surfaces are mapped and unmapped

This is done on the X11 side so we can tell when the map/unmap takes
effect and how long it took since the map request.

icon theme: Don't add profiler marks for short async icon loads

We do a bunch of preloads, and most of these are not really interesting
anyway.

builder: Don't add profiler marks for short parses

There are a lot of thes (since we're recursing), so don't spew the
output with uninteresting ones.

profiler: Add _end_mark() version of _add_mark()

These don't take a duration, instead they call g_get_monotonic_time() to
and subtract the start time for it.

Almost all our calls are like this, and this makes the callsites clearer
and avoids inlining the clock call into the call site.

profiler: Make profiler-is-running a macro

When we use if (GDK_PROFILER_IS_RUNNING) this means we get an
inlined if (FALSE) when the compiler support is not compiled in, which
gets rid of all the related code completely.

We also expand to  G_UNLIKELY(gdk_profiler_is_running ()) in the supported
case which might cause somewhat better code generation.

Convert all profiler times from nsec to usec

usec is the scale of the monotonic timer which is where we get almost
all the times from. The only actual source of nsec is the opengl
GPU time (but who knows what the actual resulution of that is).

Changing this to usec allows us to get rid of " * 1000" in a *lot* of
places all over the codebase, which are ugly and confusing.

Use markf in one more place

fixup! Add gdk_profiler_add_markf() to do printf formating

Use the new gdk_profiler_add_markf() function

Add gdk_profiler_add_markf() to do printf formating

This allows us to avoid hand-rolling g_strdup_printf calls,
but also moves the printf into the called function where
it doesn't bloat the code of the calling function if the profiler
is not running.

Update Esperanto translation

Merge branch 'mark-events-with-kind' into 'master'

profiler: Always mark events with the kind

See merge request GNOME/gtk!1414

profiler: Always mark events with the kind

Merge branch 'wip/baedert/outline-radius' into 'master'

css boxes: Compute box from border box

Closes #2425

See merge request GNOME/gtk!1408

Merge branch 'share-gl-programs' into 'master'

GskGLRenderer: Share programs between different renderers in same display

See merge request GNOME/gtk!1413

Merge branch 'handle-max-texture-size' into 'master'

gl: Don't require too large texture for offsreen ops

See merge request GNOME/gtk!1411

Merge branch 'fix-angle-normalization' into 'master'

gsk: Fix angle normalization

See merge request GNOME/gtk!1409

GskGLRenderer: Share programs between different renderers in same display

This is similar to how we share texture atlases. Some added complexity
in that the program state also needed to be shared, so it had to move to
the shared Programs object.

With this change realization of additional GskRenderers when opening
popups went from ~60msec to ~35 msec on average.

Merge branch 'ci-jobs' into 'master'

Update CI jobs

See merge request GNOME/gtk!1412

ci: Allow release job to fail

We need to investigate release-specific failures, and possibly fix our
test suite to deal with debug-specific poking through the internal
state.

For the time being, allow the release job to fail, but this is strictly
a temporary measure.

testsuite/gtk: Add xfail tests

Some tests are expected to fail.

Some tests are expected to fail in non-debugging builds.

The icontheme test is the latter.

vulkan: Handle VK_ERROR_UNKNOWN

A newly added Vulkan 1.2 error.

ci: Add a style check pass

We should at least try to get the style check done by a machine, as
opposed to have humans go through merge requests with coding style
nitpicks.

ci: Generate the report for the release builds

We are going to need to hide the report generation into the test runner
script, as we want the job to produce the reports even in case of
failure, instead of bailing out immediately.

Declare global counters only in debug builds

Silence compiler warnings in non-debug builds

Declare global counter only in debug builds

Merge branch 'fix-revealer-scale' into 'master'

revealer: Fix child size allocation at small scales

See merge request GNOME/gtk!1410

ci: Add missing newline escape

Otherwise we're going to run the accessibility test suite, and it's
going to fail.

ci: Remove static build job

Now that we figured out why the build was failing on Fedora when the
profiling code was enabled, we can go back to building both shared and
static libraries by default.

ci: Use the new Fedora image